Skip to content

Desktop: Directly upload frame buffer#2930

Merged
TrueDoctor merged 4 commits into
masterfrom
directly_upload_frame_buffer
Jul 25, 2025
Merged

Desktop: Directly upload frame buffer#2930
TrueDoctor merged 4 commits into
masterfrom
directly_upload_frame_buffer

Conversation

@TrueDoctor
Copy link
Copy Markdown
Member

@TrueDoctor TrueDoctor commented Jul 24, 2025

Partly closes #2541.

Bypass copying the frame buffer data to an intermediate vec for improved performance.
image

@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) July 24, 2025 16:24 Inactive
@TrueDoctor TrueDoctor force-pushed the directly_upload_frame_buffer branch from 3ce88dc to 99e906c Compare July 24, 2025 16:51
@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) July 24, 2025 17:19 Inactive
@timon-schelling
Copy link
Copy Markdown
Member

after disabling gpu the correct way, I get perfect frames and chromium browser level performance and responsiveness.

@timon-schelling timon-schelling force-pushed the directly_upload_frame_buffer branch from e2c9fce to 1249236 Compare July 24, 2025 19:03
@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) July 24, 2025 19:16 Inactive
@TrueDoctor TrueDoctor marked this pull request as ready for review July 24, 2025 22:08
@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) July 24, 2025 22:12 Inactive
Comment thread desktop/src/main.rs Outdated
Comment thread desktop/src/app.rs
_ui_frame_buffer: Option<wgpu::Texture>,
window_size_sender: Sender<WindowSize>,
_viewport_frame_buffer: Option<FrameBuffer>,
_viewport_frame_buffer: Option<wgpu::Texture>,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will not be used In the future right?
I think you should remove both frame buffers

@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) July 25, 2025 08:18 Inactive
@adamgerhant adamgerhant changed the title Directly upload frame buffer Desktop: Directly upload frame buffer Jul 25, 2025
@TrueDoctor TrueDoctor force-pushed the directly_upload_frame_buffer branch from 5cce137 to 7b4c54d Compare July 25, 2025 09:48
@TrueDoctor TrueDoctor force-pushed the directly_upload_frame_buffer branch from 7b4c54d to c64f4c1 Compare July 25, 2025 09:50
@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) July 25, 2025 10:17 Inactive
@TrueDoctor TrueDoctor merged commit a52ee70 into master Jul 25, 2025
4 checks passed
@TrueDoctor TrueDoctor deleted the directly_upload_frame_buffer branch July 25, 2025 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Get the viewport to render beneath the web UI

2 participants